Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

user/listenbrainz-mpd: new package #3517

Merged
merged 1 commit into from
Jan 25, 2025

Conversation

JetpackJackson
Copy link
Contributor

Description

This adds a new package called listenbrainz-mpd, which submits songs played by mpd to ListenBrainz for the user to track what songs they have listened to. This is my first contribution, so if I have made any errors, please let me know so that I can fix them ASAP.

Before submitting, I tested that the package built properly, that it worked in a Chimera Linux VM (ie it submitted songs using a working config file), and I formatted the template.py file with Black.

Checklist

Before this pull request is reviewed, certain conditions must be met.

The following must be true for all changes:

The following must be true for template/package changes:

  • I have read Packaging.md
  • I have built and tested my changes on my machine

The following must be true for new package submissions:

  • I will take responsibility for my template and keep it up to date

@q66
Copy link
Member

q66 commented Jan 24, 2025

squash your commits and rebase your branch

@JetpackJackson
Copy link
Contributor Author

Alright, will do ASAP when I get home.

@JetpackJackson
Copy link
Contributor Author

This is embarrassing, I think I messed up the squash and rebase... I hope I didn't break anything, if I did break something I want to fix it ASAP... :(

@q66
Copy link
Member

q66 commented Jan 25, 2025

you can just fix your branch and then push --force into it, starting over does not get you anything

@q66 q66 merged commit c37e2cf into chimera-linux:master Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants